Skip to content
This repository has been archived by the owner on Oct 27, 2018. It is now read-only.

Including stylesheet leads to warning in latest chrome #35

Merged
merged 2 commits into from
Oct 16, 2014
Merged

Including stylesheet leads to warning in latest chrome #35

merged 2 commits into from
Oct 16, 2014

Conversation

piscis
Copy link
Contributor

@piscis piscis commented Aug 18, 2014

Including the latest bootstrap chosen stylesheet in latest chrome, leads to the following warning in developer console:

Consider using 'dppx' units, as in CSS 'dpi' means dots-per-CSS-inch, not dots-per-physical-inch, 
so does not correspond to the actual 'dpi' of a screen. In media query expression: only screen and 
(-webkit-min-device-pixel-ratio: 2), only screen and (min-resolution: 144dpi)

@alxlit
Copy link
Owner

alxlit commented Aug 19, 2014

I'm fine with this, does it work in ie/ff?

@ben-eb
Copy link

ben-eb commented Sep 10, 2014

I've ran into this issue too. 👍

You can also suppress the warning by including the dppx unit as well as the dpi unit for greater compatibility.

@media only screen and (-webkit-min-device-pixel-ratio: 2),
       only screen and (min-resolution: 144dpi),
       only screen and (min-resolution: 2dppx) {
    // ...
}

http://css-tricks.com/snippets/css/retina-display-media-query/

alxlit pushed a commit that referenced this pull request Oct 16, 2014
Merge pull request #35 from piscis/master
61ed407 
Including stylesheet leads to warning in latest chrome
@alxlit alxlit merged commit 61ed407 into alxlit:master Oct 16, 2014
@alxlit
Copy link
Owner

alxlit commented Oct 16, 2014

thanks! apologies for the delay

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@piscis @alxlit @ben-eb